Plane-EE: Update Plane version to v1.9.3 and add email service configuration in README#107
Plane-EE: Update Plane version to v1.9.3 and add email service configuration in README#107
Conversation
|
Warning Rate limit exceeded@akshat5302 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 30 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (4)
WalkthroughThis update increments the Plane version from 1.9.2 to 1.9.3 across chart metadata, documentation, and configuration files. It introduces a configurable image pull policy for the email service deployment and documents new email service deployment options in the Helm chart README. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Helm
participant Kubernetes
participant EmailService
User->>Helm: Install/Upgrade Plane Chart (v1.9.3)
Helm->>Kubernetes: Apply manifests (with email service config)
Kubernetes->>EmailService: Deploy container (with imagePullPolicy)
EmailService-->>Kubernetes: Pull image (Always or as configured)
Possibly related PRs
Suggested reviewers
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
charts/plane-enterprise/Chart.yaml(1 hunks)charts/plane-enterprise/README.md(4 hunks)charts/plane-enterprise/questions.yml(1 hunks)charts/plane-enterprise/templates/workloads/email.deployment.yaml(1 hunks)charts/plane-enterprise/values.yaml(2 hunks)
🔇 Additional comments (8)
charts/plane-enterprise/questions.yml (1)
23-23: Plane version default updatedThe
planeVersiondefault is correctly bumped tov1.9.3, matching theappVersioninChart.yaml.charts/plane-enterprise/values.yaml (2)
1-1: Plane version updatedThe top-level
planeVersionis nowv1.9.3, aligning values with the chart metadata and questions default.
160-160: Add email service pullPolicyIntroducing
services.email_service.pullPolicy: Alwaysensures that the email service image is refreshed on each deployment, consistent with other services’ pull policies.charts/plane-enterprise/templates/workloads/email.deployment.yaml (1)
50-50: Set imagePullPolicy for email containerThe
imagePullPolicyis now driven byservices.email_service.pullPolicywith a default of"Always", matching the pattern used across other service deployments.charts/plane-enterprise/README.md (4)
14-14: Update example command Plane versionThe example export now uses
PLANE_VERSION=v1.9.3, which aligns with the chart and values changes.
68-68: Update minimum setup valuesThe required
planeVersionin the quick-start bullet list is correctly updated tov1.9.3.
103-103: Refresh license table entryThe license table now references
planeVersion | v1.9.3, consistent with other documentation updates.
289-302: Document Email Service DeploymentThe new "Email Service Deployment" section clearly outlines the
services.email_service.*keys, includingpullPolicy, and the requiredenv.email_service_envs.smtp_domain. The note about automatic TLS cert-issuer creation is informative.
charts/plane-enterprise/Chart.yaml
Outdated
| version: 1.2.3 | ||
| appVersion: "1.9.2" | ||
| appVersion: "1.9.3" |
There was a problem hiding this comment.
Chart version bump missing
You've updated the appVersion to "1.9.3" but left the chart version at 1.2.3. According to Helm best practices, the chart version should be incremented with every change to chart metadata or defaults to allow proper version tracking and publishing.
Apply this diff:
-version: 1.2.3
+version: 1.2.4📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| version: 1.2.3 | |
| appVersion: "1.9.2" | |
| appVersion: "1.9.3" | |
| version: 1.2.4 | |
| appVersion: "1.9.3" |
🤖 Prompt for AI Agents
In charts/plane-enterprise/Chart.yaml at lines 8 to 9, the chart version is
still set to 1.2.3 despite updating the appVersion to 1.9.3. To fix this,
increment the chart version to a higher value (e.g., 1.2.4) to reflect the
change in chart metadata, ensuring proper version tracking and compliance with
Helm best practices.
…md, and values.yaml
2 participants
Description
Type of Change
Screenshots and Media (if applicable)
Test Scenarios
References
Summary by CodeRabbit
New Features
Improvements
Documentation